Skip to content

public client documentation#129

Merged
sagikazarmark merged 1 commit intodexidp:mainfrom
leonnicolas:patch-2
Apr 20, 2023
Merged

public client documentation#129
sagikazarmark merged 1 commit intodexidp:mainfrom
leonnicolas:patch-2

Conversation

@leonnicolas
Copy link
Contributor

The PRs dexidp/dex#1784 and dexidp/dex#1822 are super useful, but the documentation can be confusing.

First, it does not make sense to specify a secret in a public client because it will require you to pass the secret to the public client, which will make the secret "not secret".

Also, as of dexidp/dex#1822, it is possible to use redirectURIs in a public client.

Signed-off-by: leonnicolas 60091705+leonnicolas@users.noreply.github.com

The PRs dexidp/dex#1784 and dexidp/dex#1822 are super useful, but the documentation can be confusing.

First, it does not make sense to specify a secret in a public client because it will require you to pass the secret to the public client, which will make the secret "not secret".

Also, as of dexidp/dex#1822, it is possible to use `redirectURIs` in a public client.

Signed-off-by: leonnicolas <60091705+leonnicolas@users.noreply.github.com>
Copy link
Member

@sagikazarmark sagikazarmark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@sagikazarmark sagikazarmark merged commit 2753049 into dexidp:main Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants